Skip to content

Comments

Loot Solver Edgecase and Bug Fix#91

Merged
freyamade merged 6 commits intomainfrom
loot-solver-remove-fix
Jan 9, 2025
Merged

Loot Solver Edgecase and Bug Fix#91
freyamade merged 6 commits intomainfrom
loot-solver-remove-fix

Conversation

@freyamade
Copy link
Member

Loot Solver Algorithm Improvement / Bugfix

  • Improved Loot Solver algorithm to handle newfound edgecase.
  • Also fixed silly bug in the code that was introduced when fixing the previous Loot Solver bug!

@sentry
Copy link

sentry bot commented Jan 9, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: backend/api/views/loot_solver.py

Function Unhandled Issue
_get_third_floor_data AttributeError: 'NoneType' object has no attribute 'remove' /backend/api/team/{team_id}/l...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

@freyamade freyamade merged commit 7f3fbe2 into main Jan 9, 2025
3 of 4 checks passed
@freyamade freyamade deleted the loot-solver-remove-fix branch January 9, 2025 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant